-
Notifications
You must be signed in to change notification settings - Fork 1
Backport "Check exhaustivity of any case class" to 3.3 LTS #275
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
cbac513
to
3ec2223
Compare
4dbff7f
to
8b2f2c3
Compare
No regressions detected in the community build up to backport-lts-3.3-22604. |
case (`passedEnd`, r: U @unchecked) => (thisElem, r) | ||
case (l: A @unchecked, `passedEnd`) => (l, thatElem) | ||
case t: (A, U) @unchecked => t // false-positive warning | ||
case (t1, t2) => (t1, t2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The actual code in Akka/Pekko doesn't have the case (t1, t2) => (t1, t2)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverting in #582 and for the next RC scala#23943
Revert "Merge pull request #275 from scala/backport-lts-3.3-22604"
Backports scala#22604 to the 3.3.7.
PR submitted by the release tooling.